Skip to content

chore(summary): drop decorative per-step icons in Winding Construction Steps & Termination#7

Merged
gpitel merged 1 commit into
mainfrom
chore/summary-remove-step-icons
Jun 25, 2026
Merged

chore(summary): drop decorative per-step icons in Winding Construction Steps & Termination#7
gpitel merged 1 commit into
mainfrom
chore/summary-remove-step-icons

Conversation

@gpitel

@gpitel gpitel commented Jun 25, 2026

Copy link
Copy Markdown
Owner

What

Removes the decorative per-row icons from the Winding Construction Steps and Winding Termination sections of the magnetic summary/datasheet (MagneticSummary.vue). They repeated the same glyph on every row and added nothing beyond the text label.

Removed:

  • The per-step leading icons: fa-rotate / fa-tape (construction) and fa-plug (termination).
  • The fa-shield-halved glyph inside the Primary/Secondary isolation badge (the text label stays).
  • The now-unused icon fields on the windingConstructionSteps / windingTerminations step objects, plus the .step-icon styles (including the print-stylesheet rule).

Kept (unchanged):

  • The numbered step badge, the step description, and the isolation-side text (Primary / Secondary).
  • The section-header icons (fa-list-check, fa-plug), so these sections stay visually consistent with every other section of the datasheet.

Before / after (a construction row)

Before:  [1]  (wind icon)  Wind 1-2: 24 turns                 [shield] Primary
After:   [1]  Wind 1-2: 24 turns                                       Primary

Notes

  • Net change is +2 / −29 lines, confined to MagneticSummary.vue.
  • The file has pre-existing mixed CRLF/LF line endings; the edit was applied at the byte level so only the icon lines change — no line-ending churn in the diff.
  • Verified statically: no remaining references to step.icon / .step-icon / fa-shield-halved, and the affected object literals/template/CSS remain well-formed. Not exercised in a running dev server (the running instance is on a different branch); happy to capture a live screenshot if useful.

…ination

The per-row glyphs in Winding Construction Steps (fa-rotate / fa-tape) and
Winding Termination (fa-plug) repeat the same icon on every row and add no
information beyond the text, and the shield glyph in the Primary/Secondary
isolation badge is likewise decorative. Remove them along with the now-unused
`icon` fields and `.step-icon` styles (including the print-stylesheet rule).

The numbered step, description, and isolation-side text are unchanged, and the
section-header icons are kept for consistency with the rest of the datasheet.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@gpitel gpitel merged commit 3a8e60a into main Jun 25, 2026
0 of 2 checks passed
gpitel pushed a commit that referenced this pull request Jun 26, 2026
Resolve the MagneticSummary.vue conflict by taking main's version (drops
decorative per-step icons (#7), formatting restore, schematic additions,
isolation-side labels) and re-applying the --bs-* -> --p-* theme-token
rename (#8) on top. CSS-only; line endings preserved. main.js auto-merged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant